Skip to content

feat: Add default Guzzle configuration for API client#37

Closed
tofers wants to merge 4 commits into
BushlanovDev:masterfrom
tofers:patch-2
Closed

feat: Add default Guzzle configuration for API client#37
tofers wants to merge 4 commits into
BushlanovDev:masterfrom
tofers:patch-2

Conversation

@tofers

@tofers tofers commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

По умолчанию SSL игнорируется, чтобы не вызывало ошибку.
Теперь вы можете передать свой SSL, а так же указать другие параметры если вы не используете свой client

// Включить проверку SSL-сертификатов
$api = new Api(
    'YOUR_BOT_API_TOKEN',
    guzzleConfig: ['verify' => true]
);

// Использовать свой сертификат
$api = new Api(
    'YOUR_BOT_API_TOKEN',
    guzzleConfig: ['verify' => '/path/to/certificate.crt']
);

// Настроить таймауты и прокси
$api = new Api(
    'YOUR_BOT_API_TOKEN',
    guzzleConfig: [
        'timeout' => 60,
        'proxy' => 'tcp://proxy.example.com:8080'
    ]
);

@BushlanovDev

Copy link
Copy Markdown
Owner

Приветствую!
Спасибо за реквест, но вынужден отклонить
У меня на самом деле были примерно такие же мысли на первых парах что бы не париться с сертификатом, но в итоге отказался от них
Я конечно понимаю что переписку в махе и так сливают всем заинтересованным лицам, но отключать ssl по умолчанию в и без того дырявом протоколе рука не поднялась
Передавать газл конфиг тоже избыточно, потому что можно сразу передать целиком клиент с нужными параметрами

@tofers

tofers commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Согласен, логично, но тогда из коробки библиотека не работает. Требует сертификат.

Не критично

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants